-
Notifications
You must be signed in to change notification settings - Fork 724
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Direct CLI plutus testing #2900
Conversation
98b5a96
to
5753ef6
Compare
5753ef6
to
f21b0ef
Compare
@@ -101,8 +101,6 @@ test-suite cardano-testnet-tests | |||
, hedgehog | |||
, hedgehog-extras | |||
, filepath | |||
, lens | |||
, lens-aeson |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
f21b0ef
to
a111c05
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking really good! Good work! A few comments.
|
||
-- Querying UTxO at $dummyaddress. If there is ADA at the address the Plutus script successfully executed! | ||
|
||
result <- T.pack <$> H.execCli' execConfig |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
a111c05
to
4af687f
Compare
All comments addressed. Merging. |
bors merge |
Build failed: |
5149b4b
to
0151f86
Compare
bors merge |
Build failed: |
let plutusRequiredSpace = id @Integer 70000000 | ||
let plutusRequiredTime = id @Integer 70000000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not (700000 :: Integer ) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
bors merge |
Build failed: |
e136b51
to
1d70176
Compare
1d70176
to
1e577e8
Compare
bors merge |
Build succeeded: |
This introduces a test Plutus transaction test which does not use an intervening test. Doing this directly from Haskell allows us more flexibility to make assertions.